Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
|
Caution Review failedFailed to post review comments Warning
|
| Cohort / File(s) | 설명 |
|---|---|
컨트롤러 경로 단순화 backend/src/main/java/moadong/club/controller/ClubApplyAdminController.java |
getClubApplications 시그니처에서 mode 파라미터 제거; 항상 getClubApplicationForms(user) 반환하도록 분기 삭제. |
서비스 리팩토링 backend/src/main/java/moadong/club/service/ClubApplyAdminService.java |
그룹화 관련 메서드(getGroupedClubApplicationForms) 및 관련 helper/필드(ClubRepository 등) 제거; validateSemester의 잘못된 학기 예외를 RestApiException(ErrorCode.APPLICATION_SEMESTER_INVALID)로 변경. |
리포지토리 변경 backend/src/main/java/moadong/club/repository/ClubApplicationFormsRepository.java |
ClubApplicationFormSlim 프로젝션 및 정렬 기반 조회 메서드(findClubApplicationFormsByClubId(String, Sort)) 삭제, 관련 임포트 제거. |
DTO 제거 backend/src/main/java/moadong/club/payload/dto/ClubApplicationFormSlim.java |
ClubApplicationFormSlim 인터페이스(프로젝션) 파일 삭제. |
에러 코드 추가 backend/src/main/java/moadong/global/exception/ErrorCode.java |
APPLICATION_SEMESTER_INVALID(HttpStatus.BAD_REQUEST, "800-7", "올바르지 않은 학기입니다.") 추가. |
프론트엔드: UI 컴포넌트 추가 frontend/src/components/application/QuestionTitle/QuestionTitle.tsx |
질문 제목 렌더링용 React 컴포넌트 QuestionTitle 추가(빌더/응답 모드, 자동 높이 조절 등). |
Sequence Diagram(s)
sequenceDiagram
autonumber
participant Client
participant Controller
participant Service
participant Repo
Note over Controller,Service: 이전 흐름 (mode 기반 분기)
Client->>Controller: GET /api/club/application?mode=server|agg
alt mode == "server"
Controller->>Service: getGroupedClubApplicationForms(user)
Service->>Repo: grouped query / 조합 로직
Repo-->>Service: grouped results
else
Controller->>Service: getClubApplicationForms(user)
Service->>Repo: 단일 조회 쿼리
Repo-->>Service: list results
end
Service-->>Controller: 응답
Controller-->>Client: 응답
Note over Controller,Service: 변경된 흐름 (단일 경로)
Client->>Controller: GET /api/club/application
Controller->>Service: getClubApplicationForms(user)
Service->>Repo: 단일 조회 쿼리
Repo-->>Service: list results
Service-->>Controller: 응답
Controller-->>Client: 응답
Estimated code review effort
🎯 3 (Moderate) | ⏱️ ~20 minutes
- 검토 포인트:
ClubApplicationFormSlim삭제로 인한 모든 참조(프론트/백 모두) 제거 여부 확인- 삭제된 리포지토리 메서드 및 서비스 그룹화 호출이 다른 곳에서 사용되는지 점검
validateSemester변경으로 인한 호출자 예외 처리(기존 IllegalArgumentException 대비) 및 테스트 영향 확인- 프론트엔드
QuestionTitle추가에 따른 스타일/상위 컴포넌트 통합 확인
Possibly related issues
- [refactor] MOA-318 성능테스트 결과를 반영하여 리팩토링 한다. #837 — 서버 모드 제거 및 관련 코드 경로 정리와 직접적으로 일치함 (모드 기반 분기 제거 관련).
Possibly related PRs
- [feature] 활성화된 동아리 지원서만 보여준다 #769 — DTO/리포지토리/컨트롤러 변경(학기/그룹화 관련)과 코드 레벨로 직접 중첩됨.
- Revert "[feature] 동아리 지원서 폼 제작 시에 학기를 선택할 수 있고 생성된 모든 지원서 폼을 학기별로 분류하여 조회할 수 있다" #750 — 학기/그룹화 관련 타입 및 로직 제거 이력이 겹침.
- [refactor] 클럽 지원 양식 불러오기 구버전 api를 제거한다 #793 — 동아리 신청 폼 API 및 리포지토리 변경을 다루며 중복 변경 가능성 있음.
Suggested labels
🔨 Refactor, 💾 BE, 📬 API, 💻 FE
Suggested reviewers
- Zepelown
- yw6938
Pre-merge checks and finishing touches
❌ Failed checks (2 warnings)
| Check name | Status | Explanation | Resolution |
|---|---|---|---|
| Out of Scope Changes check | 새로 추가된 QuestionTitle 프론트엔드 컴포넌트는 연결된 이슈의 서버 모드 삭제 요구사항과 무관한 변경입니다. | QuestionTitle 컴포넌트 추가는 이 PR의 범위 밖입니다. 별도의 PR로 분리하거나 연결된 이슈와의 관계를 명확히 해주세요. | |
| Docstring Coverage | Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. | You can run @coderabbitai generate docstrings to improve docstring coverage. |
✅ Passed checks (3 passed)
| Check name | Status | Explanation |
|---|---|---|
| Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. |
| Title check | ✅ Passed | 제목은 PR의 주요 변경 사항인 서버 모드 삭제와 에러코드 추가를 명확하게 요약하고 있습니다. |
| Linked Issues check | ✅ Passed | 성능 테스트 결과 반영을 위한 리팩토링 목표에 부합하게 서버 모드 제거와 에러코드 추가 구현이 완료되었습니다. |
✨ Finishing touches
- 📝 Generate docstrings
🧪 Generate unit tests (beta)
- Create PR with unit tests
- Post copyable unit tests in a comment
- Commit unit tests in branch
refactor/#837-server-mode-delete-MOA-318
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands and usage tips.
lepitaaar
approved these changes
Nov 12, 2025
Contributor
lepitaaar
left a comment
lepitaaar
left a comment
There was a problem hiding this comment.
기능개발부터 성능측정까지 수고하셨습니다!
Test Results78 tests 75 ✅ 14s ⏱️ Results for commit 01fe892. ♻️ This comment has been updated with latest results. |
lepitaaar
force-pushed
the
refactor/#837-server-mode-delete-MOA-318
branch
from
a811f8e to
01fe892
Compare
Collaborator
|
수고하셨습니당 |
yw6938
approved these changes
Nov 13, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#️⃣연관된 이슈
#837
📝작업 내용
/api/club/application api에서
기존 성능테스트를 위해 남겨두었던 server 모드를 삭제하였습니다.
지원서 생성 시, 잘못된 학기를 선택했을 때 반환될 에러코드를 구현했습니다.
중점적으로 리뷰받고 싶은 부분(선택)
논의하고 싶은 부분(선택)
🫡 참고사항
Summary by CodeRabbit
릴리스 노트
Refactor
Bug Fixes
New Features